Skip to content

Ability to use existing data #103

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
phroggyy wants to merge 21 commits into
base: master
Choose a base branch
from
Open

Ability to use existing data #103

phroggyy wants to merge 21 commits into from

Conversation

@phroggyy
Copy link

@phroggyy phroggyy commented May 23, 2015

This resolves #97. Currently, the command for using existing data rather than new data is model:factoryName, as compared to the default factory:factoryName. If there is no existing data, an entry will be created in the same fashion as factory:factoryName.

This means that, given the following conditions (and assuming we have our relations correctly defined):

$factory('App\User', [
    'first_name'     => $faker->firstName,
    'last_name'     => $faker->lastName,
    'email'             => $faker->unique()->email,
    'password'      => bcrypt($faker->word),
]);

$factory('App\Post', 'post_with_existing_user', [
    'user_id'       => 'model:App\User',
    'title'                => $faker->sentence,
    'body'              => $faker->lastName,
]);

then

Factory::create('post_with_existing_user');
Will produce a result identical to

Factory::create('App\User');
Factory::create('post_with_existing_user');

phroggyy added 21 commits May 23, 2015 19:04
@phroggyy
added a method for retrieving all indices
05fdae9
@phroggyy
added a method to retrieve a random index of an existing entry, using…
21ef8d5 
… model:FactoryName instead of factory:FactoryName for the foreign key
@phroggyy
added comment for existing post to test the ability to use an existin…
eb492e9 
…g entry. The related model is currently not eager-loaded, I think, so I expect errors in further testing...
@phroggyy
using $this for testing and added test to check if it can establish a…
d638bb1 
… relationship to an existing model
@phroggyy
corrected preg matching
29c8af7
@phroggyy
fixed allIndices
4bafcab
@phroggyy
added random method to interface
1df1af9
@phroggyy
added random and exists methods
97d8602
@phroggyy
added random and get random methods
1b9f7d2
@phroggyy
removed existingIndex and made exists public
21d3da1
@phroggyy
allowed overwriting attributes of existing posts
f357386
@phroggyy
added new factories for testing purposes
a2472a5
@phroggyy
added existing keys to support excluding related models that have alr…
fe153e0 
…eady been assigned (since a `factory:factoryName` attribute will create a new entry of `factoryName`, which might otherwise be taken by `model:factoryName`)
@phroggyy
now exclude the already taken keys when fetching a random existing model
9cb9edb
@phroggyy
added existing keys to interface
70826e4
@phroggyy
ensured a new instance is created if the user used the `model:factory…
046b57a 
…Name` attribute without having existing data
@phroggyy
set a default timezone to avoid errors about using get_default_timezo…
391d4da 
…ne()
@phroggyy
added the post_by_existing_person factory
f9483b6
@phroggyy
added test case to check if it still works multiple times when using …
3edf5f1 
…existing data
@phroggyy
added docblocks
a1a3be3
@phroggyy
added a test to assert that we never receive the same related child e…
23b5a92 
…ntity for two relationships on one parent entity
@phroggyy phroggyy mentioned this pull request May 23, 2015
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Ability to specify relation create new or use existing

1 participant

@phroggyy